Context

Fix the issue in the previous PR: #21200
Failed pipeline run link: https://dev.azure.com/mseng/AzureDevOps/_build/results?buildId=30323883&view=logs&s=7cadbae7-7533-548a-becd-a2da10fc0146

Task Name

NA

Description

Key changes include:

• Removed AzureDevOps repository from pipeline resources to prevent CG dependency conflicts
• Implemented repository checkout logic within courtesy-push.js
• Added a dryRun parameter to avoid pushing of nuget artifacts into the feed and also avoid PR creation
• The display name of the publish job is changed to 'Publish Nuget Artifacts'
• Added access token to clone the AzureDevOps repo

Risk Assessment (Low / Medium / High)

Low

Change Behind Feature Flag (Yes / No)

No because this is a release pipeline.

Tech Design / Approach

The original pipeline approach included AzureDevOps repository as a pipeline resource, which created CG dependency conflicts. Additionally, there was no safe testing mechanism for validating the entire workflow without external side effects.

Changes made:

• Eliminated the AzureDevops repository resource from azure-pipelines.yml and implemented the checkout logic within courtesy-push.js
• Added a dryRun parameter which allows to run all the steps but skips actual pushing of Nuget artifacts to the feed and also skips pushing release notes created in the azure-pipelines-tasks repo. This also skips creation of PR for the changes made in the unified_deps.xml file

Documentation Changes Required (Yes/No)

Yes.
Added a new paramter dryRun in the courtesy push pipeline.

Image:

Unit Tests Added or Updated (Yes / No)

No - This is an internal refactor of existing functionality. The courtesy push logic remains functionally equivalent, and comprehensive testing will be performed through integration testing in the pipeline environment

Additional Testing Performed

Locally tested:

• Verified repository cloning logic with various environment variable configurations.
• Validated temp directory path resolution in Azure DevOps agent environment
• Confirmed dryRun parameter correctly skips actual PR creation steps

Logging Added/Updated (Yes/No)

Yes

• Added detailed logging for repository path determination and environment variable detection
• Enhanced git operation logging with status checks before and after operations
• Improved error messaging for authentication and repository access issues
  Link: https://dev.azure.com/surajitshil/Azure-Repo-File-Structure/_build/results?buildId=289&view=logs&j=12f1170f-54f2-53f3-20dd-22fc7dff55f9&t=9c939e41-62c2-5605-5e05-fc3554afc9f5

Telemetry Added/Updated (Yes/No)

No

Rollback Scenario and Process (Yes/No)

NA

Dependency Impact Assessed and Regression Tested (Yes/No)

NA

Checklist

NA